* Fix oddity with open tag parameters getting stuck on </li>

diff --git a/RELEASE-NOTES b/RELEASE-NOTES
index 8d7fbe2..0eb51ed 100644 (file)
--- a/RELEASE-NOTES
+++ b/RELEASE-NOTES
@@ -397,6 +397,7 @@ it from source control: http://www.mediawiki.org/wiki/Download_from_SVN
 * When showing the "blah has been undeleted" page, make sure it's a blue link
 * parserTests.php accepts a --file parameter to run an alternate test sutie
 * parser tests can now test extensions using !!hooks sections
+* Fix oddity with open tag parameters getting stuck on </li>
 
 
 == Compatibility ==

diff --git a/includes/Sanitizer.php b/includes/Sanitizer.php
index 25f4d57..30c3a21 100644 (file)
--- a/includes/Sanitizer.php
+++ b/includes/Sanitizer.php
@@ -400,8 +400,8 @@ class Sanitizer {
                                                        if ( $t == 'table' ) {
                                                                $tagstack = array_pop( $tablestack );
                                                        }
-                                                       $newparams = '';
                                                }
+                                               $newparams = '';
                                        } else {
                                                # Keep track for later
                                                if ( in_array( $t, $tabletags ) &&

diff --git a/maintenance/parserTests.txt b/maintenance/parserTests.txt
index 5732c10..b65ded1 100644 (file)
--- a/maintenance/parserTests.txt
+++ b/maintenance/parserTests.txt
@@ -4013,6 +4013,15 @@ HTML nested ordered list, open tags (bug 5497)
 
 !! end
 
+!! test
+HTML ordered list item with parameters oddity
+!! input
+<ol><li id="fragment">One</li></ol>
+!! result
+<ol><li id="fragment">One</li></ol>
+
+!! end
+
 !!test
 bug 5918: autonumbering
 !! input